Skip to content

remove unused functions from RuntimeExecutor.h#44197

Closed
philIip wants to merge 1 commit intofacebook:mainfrom
philIip:export-D56447959
Closed

remove unused functions from RuntimeExecutor.h#44197
philIip wants to merge 1 commit intofacebook:mainfrom
philIip:export-D56447959

Conversation

@philIip
Copy link
Copy Markdown
Contributor

@philIip philIip commented Apr 23, 2024

Summary:
Changelog: [C++][Breaking] Remove executeAsynchronously and executeSynchronously_CAN_DEADLOCK

these are not used anywhere nor in OSS, we can delete this safely

Differential Revision: D56447959

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 23, 2024
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D56447959

@analysis-bot
Copy link
Copy Markdown

analysis-bot commented Apr 23, 2024

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 19,410,771 -15
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 22,784,294 -7
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: d59de54
Branch: main

Summary:

Changelog: [C++][Breaking] Remove executeAsynchronously and executeSynchronously_CAN_DEADLOCK

these are not used anywhere nor in OSS, we can delete this safely

Differential Revision: D56447959
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D56447959

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Apr 24, 2024
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request has been merged in 8b1a016.

@github-actions
Copy link
Copy Markdown

This pull request was successfully merged by @philIip in 8b1a016.

When will my fix make it into a release? | How to file a pick request?

kosmydel pushed a commit to kosmydel/react-native that referenced this pull request May 6, 2024
Summary:
Pull Request resolved: facebook#44197

Changelog: [General][Breaking] Remove executeAsynchronously and executeSynchronously_CAN_DEADLOCK

these are not used anywhere nor in OSS, we can delete this safely

Reviewed By: javache

Differential Revision: D56447959

fbshipit-source-id: d66c10f676946422385750c1b8825ead2d5d0ed8
kosmydel pushed a commit to kosmydel/react-native that referenced this pull request Jun 11, 2024
Summary:
Pull Request resolved: facebook#44197

Changelog: [General][Breaking] Remove executeAsynchronously and executeSynchronously_CAN_DEADLOCK

these are not used anywhere nor in OSS, we can delete this safely

Reviewed By: javache

Differential Revision: D56447959

fbshipit-source-id: d66c10f676946422385750c1b8825ead2d5d0ed8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants